Added multiple trial functionality, other minor changes#9
Open
Chryzanthemum wants to merge 7 commits intoroycoding:masterfrom
Chryzanthemum:master
Open
Added multiple trial functionality, other minor changes#9Chryzanthemum wants to merge 7 commits intoroycoding:masterfrom Chryzanthemum:master
Chryzanthemum wants to merge 7 commits intoroycoding:masterfrom
Chryzanthemum:master
Conversation
added 3 commits
April 10, 2018 17:37
Added info, current, multiple trial functionality, est payout by bandit
slots/slots.py
Outdated
| ''' | ||
| Feeds two arrays in and based on those results returns the next trial. | ||
| This really isn't optimized, there's a much better way of doing this if we don't | ||
| care about maintaining the |
added 4 commits
April 20, 2018 17:03
Some list/array comprehension stuff
So I combined my two multiple trial methods into a 'hard' and a 'lazy' option, and I'm also starting to track each payout value. This definitely increases the size of the object, so maybe add an option to disable that, but it's important to the next step which is to add a time-sliding scale.
Multiple Trials condense, track all payout values
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
multiple_trials - iterates over an array and uses online trial for each pair of bandits/payouts
multiple_trials_bulk - sums up array of payouts to update choices, payouts without iterating
est_payouts - added option to choose which bandit to see payouts for
info - added number of bandits, number of wins for each bandit, and estimated payouts for each (wasn't sure why you wanted to put it under the bandit class)
current - added current bandit choice (choices[-1])